Repository navigation
Conversation
…Accessor' options
|
|
View your CI Pipeline Execution ↗ for commit df04d43
☁️ Nx Cloud last updated this comment at |
🚀 Changeset Version Preview2 package(s) bumped directly, 22 bumped as dependents. 🟩 Patch bumps
|
size-limit report 📦
|
|
Important Review skippedReview was skipped as selected files did not have any reviewable changes.
|
| Layer / File(s) | Summary |
|---|---|
Reference tables and generator normalization docs/framework/lit/reference/functions/createQueryController.md, docs/framework/lit/reference/functions/createInfiniteQueryController.md, docs/framework/lit/reference/functions/createMutationController.md, docs/framework/lit/reference/functions/createQueriesController.md, docs/framework/lit/reference/functions/useIsFetching.md, docs/framework/lit/reference/functions/useIsMutating.md, scripts/generate-docs.ts |
The reference pages replace generic type references with property tables for query, mutation, and filter options. The generator strips surrounding parentheses from each union member before checking wrapper aliases. |
Estimated code review effort: 2 (Simple) | ~10 minutes
Change: Bug fix
Suggested reviewers: alemtuzlak, lachlancollins
Merge Risk: 
Minimal · previous review 2b9d8
-
This documentation-only change has no established merge-blocking risk.
Pre-merge checks: 
5
-
Full details: Docstring Coverage
-
Explanation
Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 1 functions across 1 files. (6 skipped: 6 unsupported.)
-
Finishing Touches
-
Generate docstrings
-
- Commit to this branch
- Create a new PR
🧪 Generate unit tests (beta)
-
- Commit to this branch
- Create a new PR
-
- Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.
…renthesized-wrapper-alias
🎯 Changes
isWrapperAlias(scripts/generate-docs.ts), strip the outer parentheses from each union member before comparing it toTor() => T. A union prints a function member as(() => T), so an alias like lit'sAccessor<T> = T | (() => T)was not treated as a wrapper alias and its options section only got a "Built from" link.createQueryController,createInfiniteQueryController,createMutationController,createQueriesController,useIsFetching, anduseIsMutatingnow show the full property table for theirAccessoroptions. No other adapter's docs change.✅ Checklist
pnpm run test:pr, or these tests do not apply to this pull request.🚀 Release Impact
Summary by CodeRabbit
Documentation
Chores